-
-
Notifications
You must be signed in to change notification settings - Fork 322
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Web Balance Report #2242
base: master
Are you sure you want to change the base?
Web Balance Report #2242
Conversation
6536fb2
to
3692f9a
Compare
I played around with this. Exciting to see changes in hledger-web, and especially multi-period reports! This one seems to be a balance changes report, like the With hledger-web I have always been aiming for an easy UI useful to non technical users, that's simple enough to maintain and keep robust. It doesn't look like much but it has had a lot of rough edges smoothed off (some remain..). It has also probably grown quite a bit of tech debt, and may be hard to extend in big ways without some reorg or even a fresh approach. I haven't focussed on it in some time though. I am wondering about having an experimental mode, so that we could try out new stuff more freely without creating churn for regular users. Or even a separate binary, hledger-web-x, if needed. |
PS shall we change this PR's status to draft. |
Yes, I think draft is the right status. However, it seems I cannot change the status myself. |
125bf88
to
80af57a
Compare
78474cc
to
6fd8f42
Compare
6521c18
to
8390fa4
Compare
8390fa4
to
7890661
Compare
7890661
to
38611ae
Compare
…od balance reports as HTML pages Journal page contains links to those reports.
38611ae
to
2655258
Compare
This is a preview for a new feature of hledger-web.
It adds serving balance reports from hledger-web.
So far there remain some issues:
defaultLayout
, which means there is always the sidebar. Do we want that? Of course, the user can hide the sidebar withs
key. However, I found the query field useful in conjunction with the balance reports.Journal
does not maintain the current query. In the past I did it this way in order to have quick way out of restrictions posed by queries. However, all other accounts in the sidebar in the query and the hyperlinks behind dates in transactions in theRegister
page also maintain the query. Since I currently have the links to balance reports on the Journal page it would be nice if the top Journal link in the sidebar would also contain the query.